-
-
Notifications
You must be signed in to change notification settings - Fork 45
Deprecation warnings on PHP 8.4 #356
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
composer.lock
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't commit the composer lock file, have removed so the tests.yml will work
| "orchestra/testbench": "^8.0 || ^9.0 || ^10.0", | ||
| "laminas/laminas-code": "^4.8.0", | ||
| "nunomaduro/collision": "^8.1", | ||
| "symplify/easy-ci": "^10.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is abandoned now
| /** | ||
| * Class Soap. | ||
| * | ||
| * @method static \CodeDredd\Soap\Handler\HttPlugHandle getHandler() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missing file so removed reference
|
@CodeDredd Are you able to take a look at this one and give me any feedback please. Thank you |
|
@CodeDredd any idea when you'll have time for this one? |
Issue: Deprecation warnings on PHP 8.4
What I did
Changed the deprecated sections and also got tests working. The versions we allow for php-soap/psr18-transport only support php 8.2 upwards. So i adjusted tests
How to test
I have made the changes to match the new requirements
If your answer is yes to any of these, please make sure to include it in your PR.